Skip to content

Use pebble instead of mustache#1021

Open
Yang-33 wants to merge 5 commits into
masterfrom
pebble
Open

Use pebble instead of mustache#1021
Yang-33 wants to merge 5 commits into
masterfrom
pebble

Conversation

@Yang-33

@Yang-33 Yang-33 commented Jun 20, 2026

Copy link
Copy Markdown
Contributor

Almost all line-bot-sdk-* uses pebble as template engine instead of mustache.
This change migrates into it, because we'd like to use same way, and pebble is richer and easier than mustache.

Main diff is linebot/v3. There is no big impact at all.

Comment on lines -584 to -592
:param grant_type: `client_credentials` (required)
:param grant_type: `client_credentials`
:type grant_type: str
:param client_assertion_type: URL-encoded value of `urn:ietf:params:oauth:client-assertion-type:jwt-bearer` (required)
:param client_assertion_type: URL-encoded value of `urn:ietf:params:oauth:client-assertion-type:jwt-bearer`
:type client_assertion_type: str
:param client_assertion: A JSON Web Token the client needs to create and sign with the private key of the Assertion Signing Key. (required)
:param client_assertion: A JSON Web Token the client needs to create and sign with the private key of the Assertion Signing Key.
:type client_assertion: str
:param client_id: Channel ID. (required)
:param client_id: Channel ID.
:type client_id: str
:param client_secret: Channel secret. (required)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fix is correct. It is not required, and the previous implementation was simply incorrect.

"""
AUDIENCE_GROUP_AUDIENCE_INSUFFICIENT = 'AUDIENCE_GROUP_AUDIENCE_INSUFFICIENT'
INTERNAL_ERROR = 'INTERNAL_ERROR'
NULL = 'null'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actually null as string is not returned from Messaging API.

https://developers.line.biz/en/reference/messaging-api/#get-shared-audience

Image

openapi schema mistakenly contains this, and some sdks defines NULL as string...

line/line-openapi#124 fixes it.

@Yang-33 Yang-33 marked this pull request as ready for review June 20, 2026 03:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant